New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[no-op] Resolve switch case warning #12

Open

dentiny wants to merge 1 commit into duckdb:main from dentiny:hjiang/resolve-switch-case-warning

dentiny commented Jan 21, 2025

I included httpfs files inside of my project, but it fails compilation due to my compilation option.

Error message:

../../third_party/duckdb/extension/httpfs/crypto.cpp:139:1: error: control reaches end of non-void function [-Werror=return-type]
  139 | }
      | ^
cc1plus: all warnings being treated as errors


          resolve switch case warning

Author

dentiny commented Jan 24, 2025

Hi @samansmink do you mind taking a look when you have some time? This PR should be an easy no-op change. Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet